Skip to content

[KeyVault] Fix flaky tests around the getDeleted* tests#4688

Merged
sadasant merged 3 commits intomasterfrom
feature/KeyVault-fixFlakyTests
Aug 8, 2019
Merged

[KeyVault] Fix flaky tests around the getDeleted* tests#4688
sadasant merged 3 commits intomasterfrom
feature/KeyVault-fixFlakyTests

Conversation

@sadasant
Copy link
Copy Markdown
Contributor

@sadasant sadasant commented Aug 6, 2019

The issue is likely to be that we weren't waiting for both entities to be deleted before we moved on.

The issue is likely to be that we weren't waiting for both entities to be deleted before we moved on.
@sadasant sadasant added Client This issue points to a problem in the data-plane of the library. KeyVault labels Aug 6, 2019
@sadasant sadasant requested a review from sophiajt August 6, 2019 17:36
@sadasant sadasant self-assigned this Aug 6, 2019
@sadasant
Copy link
Copy Markdown
Contributor Author

sadasant commented Aug 7, 2019

/azp run js - keyvault-secrets - tests

@sadasant
Copy link
Copy Markdown
Contributor Author

sadasant commented Aug 7, 2019

/azp run js - keyvault-keys - tests

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

1 similar comment
@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@sadasant
Copy link
Copy Markdown
Contributor Author

sadasant commented Aug 7, 2019

The test that failed failed because of a connection error 🤔 I restarted here: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=84837&view=results

@sadasant
Copy link
Copy Markdown
Contributor Author

sadasant commented Aug 8, 2019

I can't reproduce the build issue locally.

@sadasant
Copy link
Copy Markdown
Contributor Author

sadasant commented Aug 8, 2019

Merging for now. I'll check how the automatic builds go.

@sadasant sadasant merged commit 60e1bf9 into master Aug 8, 2019
@sadasant sadasant deleted the feature/KeyVault-fixFlakyTests branch August 8, 2019 19:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Client This issue points to a problem in the data-plane of the library. KeyVault

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants